-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for platform logs in Filebeat Azure module #22371
Conversation
thanks @andrewkroh , looking at the expected types I don't think we could match |
I think there are categories that could be applied. For example there are some |
"group" : "OBS-TEST" | ||
}, | ||
"platformlogs" : { | ||
"Status" : "Succeeded", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This field doesn't have a mapping, right? I see you use them to fill other fields, should it be dropped or mapped?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added mapping for this field
@@ -0,0 +1 @@ | |||
{"ActivityId":"30ed877c-a36b-491a-bd4d-ddd847fe55b8","Caller":"Portal","Environment":"PROD","EventName":"Retreive ConsumerGroup","EventProperties":"{\"SubscriptionId\":\"7657426d-c4c3-44ac-88a2-3b2cd59e6dba\",\"Namespace\":\"obstesteventhubs\",\"Via\":\"sb://obstesteventhubs.servicebus.windows.net/insights-logs-operationallogs/consumergroups?api-version=2017-04\u0026$skip=0\u0026$top=100\",\"TrackingId\":\"30ed877c-a36b-491a-bd4d-ddd847fe55b8_M2CH3_M2CH3_G3S2\"}","EventTimeString":"11/3/2020 9:06:42 AM +00:00","Region":"West Europe","ScaleUnit":"PROD-AM3-AZ501","Status":"Succeeded","SubscriptionId":"7657426d-c4c3-44ac-88a2-3b2cd59e6dba","category":"OperationalLogs","resourceId":"/SUBSCRIPTIONS/7657426D-C4C3-44AC-88A2-3B2CD59E6DBA/RESOURCEGROUPS/OBS-TEST/PROVIDERS/MICROSOFT.EVENTHUB/NAMESPACES/OBSTESTEVENTHUBS"} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be nice to have several lines here with different types, to give a better idea of what we can expect
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
additional example in
This is looking great! I left a few comments |
* mofidy doc * platformlogs * fix * separate pr * work on platform * work * platforms * changelog * fix file * add tests * add mapping * test * update mapping * fix file name * update file * map field * update files * fix logs * generate tests (cherry picked from commit 2ae52c3)
* mofidy doc * platformlogs * fix * separate pr * work on platform * work * platforms * changelog * fix file * add tests * add mapping * test * update mapping * fix file name * update file * map field * update files * fix logs * generate tests (cherry picked from commit 2ae52c3)
What does this PR do?
Add support for platform logs in Filebeat Azure module
Why is it important?
Platform logs provide detailed diagnostic and auditing information for Azure resources and the Azure platform they depend on.
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Example platform logs:
Ex outputs: